Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refine the chapter 2 sentences #340

Merged
merged 9 commits into from
Jan 9, 2024
Merged

Refine the chapter 2 sentences #340

merged 9 commits into from
Jan 9, 2024

Conversation

YKIsTheBest
Copy link
Contributor

Refine the chapter 2 sentences

Add english annotate for special term
Refine the sentences
Refine sentences
Refine sentence for more comprehensible in Chinese
Refine sentence for more comprehensible in Chinese
Refine sentence for more comprehensible in Chinese
Copy link
Collaborator

@yingang yingang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

而 MapReduce 的用途并非专垄于分布式查询执行上。

这一句感觉稍微有点别扭,专垄好像也不是个惯用语,建议改成“而 MapReduce 也并没有垄断所有的分布式查询执行”?

Refine the sentences
@YKIsTheBest
Copy link
Contributor Author

而 MapReduce 的用途并非专垄于分布式查询执行上。

这一句感觉稍微有点别扭,专垄好像也不是个惯用语,建议改成“而 MapReduce 也并没有垄断所有的分布式查询执行”?

Thanks for reviewing , file is updated.

@yingang
Copy link
Collaborator

yingang commented Jan 7, 2024

而 MapReduce 的用途并非专垄于分布式查询执行上。
这一句感觉稍微有点别扭,专垄好像也不是个惯用语,建议改成“而 MapReduce 也并没有垄断所有的分布式查询执行”?

Thanks for reviewing , file is updated.

好的,麻烦也请确认下另外几处建议~

@YKIsTheBest
Copy link
Contributor Author

而 MapReduce 的用途并非专垄于分布式查询执行上。
这一句感觉稍微有点别扭,专垄好像也不是个惯用语,建议改成“而 MapReduce 也并没有垄断所有的分布式查询执行”?

Thanks for reviewing , file is updated.

好的,麻烦也请确认下另外几处建议~

I didnt notice any other advises, can you tell me where it is?
Or, Would you kindly put them in the comment.

Thanks a lot!

@yingang
Copy link
Collaborator

yingang commented Jan 8, 2024

好的,麻烦也请确认下另外几处建议~

I didnt notice any other advises, can you tell me where it is? Or, Would you kindly put them in the comment.

Thanks a lot!

我也不太熟悉 Github 的 review 机制,你这边看不到类似这样的信息是吧?

image

那我先直接贴到这里吧,有三个小点:

  1. 即使只访问文档其中的一小部分,数据库通常需要加载整个文档,【这】对于大型文档来说这种加载行为是很浪费的。 -- 建议删除掉前面那个【这】,感觉更通顺一些?
  2. 【没有特定的框架须遵守】对 JSON 模型来说往往被认为是一个优势 -- 这里的schema应该是模式而不是框架,建议更新为“无模式对 JSON 模型来说往往被认为是一个优势”?
  3. (效果称为尾部延迟放大(tail latency amplification)【24】) -- 括号套括号有点不太好看,建议更新为:(该效果称为尾部延迟放大,即 tail latency amplification【24】)?

Refine sentences as advices
Refine sentences as advices
@YKIsTheBest
Copy link
Contributor Author

好的,麻烦也请确认下另外几处建议~

I didnt notice any other advises, can you tell me where it is? Or, Would you kindly put them in the comment.
Thanks a lot!

我也不太熟悉 Github 的 review 机制,你这边看不到类似这样的信息是吧?

Yes, I can't find any below comments, sorry about that.
Maybe I will figure it out sometimes.

那我先直接贴到这里吧,有三个小点:

  1. 即使只访问文档其中的一小部分,数据库通常需要加载整个文档,【这】对于大型文档来说这种加载行为是很浪费的。 -- 建议删除掉前面那个【这】,感觉更通顺一些?
  2. 【没有特定的框架须遵守】对 JSON 模型来说往往被认为是一个优势 -- 这里的schema应该是模式而不是框架,建议更新为“无模式对 JSON 模型来说往往被认为是一个优势”?
  3. (效果称为尾部延迟放大(tail latency amplification)【24】) -- 括号套括号有点不太好看,建议更新为:(该效果称为尾部延迟放大,即 tail latency amplification【24】)?

Thanks for reviewing, already modified.

@yingang yingang merged commit 4eac55b into Vonng:master Jan 9, 2024
@yingang
Copy link
Collaborator

yingang commented Jan 9, 2024

Thanks for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants